@claude 请审核这个零行为诊断 PR（#470 诊断 v2）。重点：(1) 三处 Windows-cfg log（no_activate 两处 warn、lib.rs 几何 debug）的 format 占位符与参数数量/类型是否匹配（本机无法交叉编译验证）；(2) CAPSULE_WINDOW_MISSING_LOGGED 一次性门是否正确、是否会在 ~30Hz 音频回调线程刷屏；(3) 前端 capsuleStateFirstLogged 模块级一次性门是否合理；(4) 确认确实未改任何 show/hide/position 行为。

Claude finished @appergb's task in 3m 13s —— View job

审核 PR #593 — #470 诊断 v2

• 读取变更文件
• 验证 format 占位符与参数
• 分析 CAPSULE_WINDOW_MISSING_LOGGED 一次性门
• 分析前端 capsuleStateFirstLogged 门
• 确认零行为改动
• 撰写审核报告

(1) Format 占位符 / 参数校验 ✅

coordinator.rs — show_capsule_window_no_activate 两处 warn!

两条均为纯字符串字面量，零占位符、零参数，不存在不匹配风险：

log::warn!("[capsule] no_activate failed: window_handle() unavailable — Win32 show skipped");
log::warn!("[capsule] no_activate failed: non-Win32 RawWindowHandle — Win32 show skipped");

lib.rs — position_capsule_bottom_center 几何 debug!

10 个占位符 = 10 个参数，类型全部匹配，{:.2} 对 f64 合法。✅

(2) CAPSULE_WINDOW_MISSING_LOGGED 一次性门 ✅ — 但有一处同 PR 内部不一致⚠️

门本身正确。 AtomicBool::swap(true, Ordering::SeqCst) 与文件内既有三处门（CAPSULE_NO_ACTIVATE_FALLBACK_WARNED、CAPSULE_SUPPRESSED_BY_TOGGLE_LOGGED、CAPSULE_FIRST_SHOW_LOGGED）完全一致。

线程安全 ✅ — 此门在 run_on_main_thread 闭包 内部（主线程）执行，不在音频回调线程上。音频线程只做 app.run_on_main_thread(...) 入队，swap 本身已是原子操作，即使多个闭包排队也只有第一个通过。不会 ~30 Hz 刷屏。

⚠️ 问题：同 PR 里的两处 no_activate warn! 没有类似的门

show_capsule_window_no_activate 也在 run_on_main_thread 的调用链内（emit_capsule → 主线程闭包 → show_capsule_window_for_recording → show_capsule_window_no_activate）。若 window_handle() 持续返回 Err（例如整个录音会话 HWND 都不可用），这两条 warn! 将以 ~30 Hz 频率持续打印：

// 这两条无速率限制
log::warn!("[capsule] no_activate failed: window_handle() unavailable — Win32 show skipped");
log::warn!("[capsule] no_activate failed: non-Win32 RawWindowHandle — Win32 show skipped");

外层 CAPSULE_NO_ACTIVATE_FALLBACK_WARNED（line 128）只门住 "falling back to window.show()" 那条日志，不压制这两条内层 warn。

建议修复：补一个共享的一次性门，与其他几处保持一致风格：Fix this →

// 建议在 line ~100 加：
static CAPSULE_NO_ACTIVATE_HWND_WARNED: AtomicBool = AtomicBool::new(false);

// 然后包住两条 warn：
let Ok(handle) = window.window_handle() else {
    if !CAPSULE_NO_ACTIVATE_HWND_WARNED.swap(true, Ordering::SeqCst) {
        log::warn!("[capsule] no_activate failed: window_handle() unavailable — Win32 show skipped");
    }
    return false;
};
let RawWindowHandle::Win32(raw) = handle.as_raw() else {
    if !CAPSULE_NO_ACTIVATE_HWND_WARNED.swap(true, Ordering::SeqCst) {
        log::warn!("[capsule] no_activate failed: non-Win32 RawWindowHandle — Win32 show skipped");
    }
    return false;
};

(3) 前端 capsuleStateFirstLogged 模块级门 ✅

let capsuleStateFirstLogged = false;
// ...
if (!capsuleStateFirstLogged) {
  capsuleStateFirstLogged = true;
  console.info('[capsule] first capsule:state received in webview, state=', p.state);
}

合理。 ES 模块是 webview 生命周期内的单例，let 模块变量是正确的一次性门实现。JavaScript 单线程，无竞态。相比 React useRef 或 useState，模块变量更简单，也不会因组件 unmount/remount（含 React StrictMode 开发期双调用）而重置——对诊断目的（「确认 webview 至少收到过一次事件」）语义正确。

唯一的边缘情况：若 Capsule 组件先被 unmount、等窗口重建后再 mount，后续第一次收到事件不会再打 log。但此场景极少见，且对诊断 #470 不关键。可接受。✅

(4) 零行为改动确认 ✅

确认：无任何 show/hide/position 行为改动。 ✅

总结

可以合并，有一处建议修复（非阻塞但推荐）：

• show_capsule_window_no_activate 内两条 warn! 缺一次性门，与 PR 中 CAPSULE_WINDOW_MISSING_LOGGED 的设计意图不一致，若 HWND 持续不可用会产生 ~30 Hz 日志。建议补 CAPSULE_NO_ACTIVATE_HWND_WARNED 门后再合。

其余三个审核点（format 串、CAPSULE_WINDOW_MISSING_LOGGED 门、前端门）全部正确，零行为改动确认无误。

PR Reviewer Guide 🔍

Here are some key observations to aid the review process: